Skip to content

Fix proof namespace method descriptions to match implementation#340

Draft
Copilot wants to merge 2 commits intomainfrom
copilot/fix-proof-module-description
Draft

Fix proof namespace method descriptions to match implementation#340
Copilot wants to merge 2 commits intomainfrom
copilot/fix-proof-module-description

Conversation

Copy link
Copy Markdown

Copilot AI commented May 6, 2026

The proof_getTransactionByHash and proof_getTransactionReceipt method descriptions in the docs were both inaccurate — getTransactionByHash was described as returning eth_getTransactionReceipt data, and getTransactionReceipt was described as returning eth_call data with account/storage proofs. Neither matched the actual behavior in IProofRpcModule.cs.

Changes

  • proof_getTransactionByHash: Updated description to correctly state it returns eth_getTransactionByHash data plus a Merkle-Patricia proof of the transaction's inclusion in the block's transactionsRoot, with an optional RLP-encoded block header when includeHeader is true.
  • proof_getTransactionReceipt: Updated description to correctly state it returns eth_getTransactionReceipt data plus Merkle-Patricia proofs of the transaction's inclusion in transactionsRoot and the receipt's inclusion in receiptsRoot, with an optional RLP-encoded block header when includeHeader is true.

Applied to the current docs and all versioned docs (v1.33.0–v1.37.0).

Copilot AI changed the title [WIP] Fix proof module description mismatch in documentation Fix proof namespace method descriptions to match implementation May 6, 2026
Copilot AI requested a review from MarekM25 May 6, 2026 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Description of proof module doesn't match what we have in code

2 participants